Issue 49544: Remove the ability to select specific objects to import and apply import to multiple folders under the advanced import options issues#7153
Conversation
|
WARNING: This PR appears to have the default title generated by GitHub. Please use something more descriptive. |
labkey-adam
left a comment
There was a problem hiding this comment.
Shouldn't ImportOptions._advancedImportOptions and references to it be removed?
Same question for ImportFolderForm._advancedImportOptions
Yup, missed those, now removed. Sorry thought I had done due diligence, but had to Invalidate Cache and restart IntelliJ to find all the references. Also, should have used Junie - a good use-case for AI. |
labkey-adam
left a comment
There was a problem hiding this comment.
Code changes look good, but test suites are failing with SMTP errors. I don't know why. Maybe @labkey-danield knows?
The Password test fails for me locally in this branch, but passes when I switch back to develop. |
In TestAutomation the code in DumbsterManager should not have been removed. I have a local change that reverts that change. I will check it in shortly. |
Rationale
Issue 49544: Remove the ability to select specific objects to import and apply import to multiple folders under the advanced import options.
Previously, this option was deprecated (see PR). In this PR, we are removing it entirely.
Related Pull Requests
Changes